Skip to content

Change: Apollo Client update#325

Merged
shreddedbacon merged 19 commits intomainfrom
gql-update-v3
Jul 28, 2025
Merged

Change: Apollo Client update#325
shreddedbacon merged 19 commits intomainfrom
gql-update-v3

Conversation

@CGoodwin90
Copy link
Contributor

@CGoodwin90 CGoodwin90 commented May 30, 2025

Updates apollo to be in line with the api via (uselagoon/lagoon#3921).
All query and mutation components are deprecated with this update and required replacing with the useQuery and useMutation hooks.

Requires uselagoon/lagoon#3921

@CGoodwin90 CGoodwin90 changed the title WIP: Apollo update Change: Apollo Client update Jul 10, 2025
@CGoodwin90 CGoodwin90 marked this pull request as ready for review July 10, 2025 04:19
@CGoodwin90 CGoodwin90 requested review from DaveDarsa and bomoko July 10, 2025 04:19
@bomoko bomoko requested a review from rocketeerbkw July 10, 2025 20:45
@bomoko bomoko removed their request for review July 20, 2025 20:54
Copy link
Member

@rocketeerbkw rocketeerbkw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes are primarily apollo API related and does not significantly alter functionality.

There appears to be one issue with subscriptions, where leaving a page open seems to open new subscriptions which causes data returned by the API to be duplicated for each open connection. This does not negatively impact the UI at all (the subscription logic handles the duplicates) and now that the backend is using redis, I don't see any issues there either. Considering the overall deprecation of this version of the UI, this should be tackled in the next version.

2025-07-27_11-40

@shreddedbacon shreddedbacon merged commit 71cb314 into main Jul 28, 2025
4 checks passed
@shreddedbacon shreddedbacon deleted the gql-update-v3 branch July 28, 2025 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants